workspaces/ballerina/ballerina-extension/src/features/test-explorer/runner.ts (1)

230-246: ⚡ Quick win

Run pre-test project cleanup concurrently to avoid serializing LSP round-trips.

When a run touches multiple projects (common for workspace runs — exactly the scenario this PR targets), cleanAndValidateProject is awaited one project at a time, so each LSP round-trip blocks the next. Since the per-project cleanups are independent and errors are already swallowed/logged, Promise.allSettled gets the same semantics with parallelism.

Additionally, consider checking token.isCancellationRequested before/while cleaning so users can abort during the pre-test phase (today cancellation is only honored inside the per-test forEach below).

♻️ Suggested refactor

-    const langClient = extension.ballerinaExtInstance.langClient;
-    for (const projectPath of projectPaths) {
-        try {
-            await cleanAndValidateProject(langClient, projectPath);
-        } catch (err) {
-            console.error(`Failed to clean project before test run: ${projectPath}`, err);
-        }
-    }
+    const langClient = extension.ballerinaExtInstance.langClient;
+    await Promise.allSettled(
+        Array.from(projectPaths).map(async (projectPath) => {
+            try {
+                await cleanAndValidateProject(langClient, projectPath);
+            } catch (err) {
+                console.error(`Failed to clean project before test run: ${projectPath}`, err);
+            }
+        })
+    );
+    if (token.isCancellationRequested) {
+        include.forEach((test) => run.skipped(test));
+        run.end();
+        return;
+    }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@workspaces/ballerina/ballerina-extension/src/features/test-explorer/runner.ts`
around lines 230 - 246, The current serial pre-test cleanup loops over
projectPaths and awaits cleanAndValidateProject for each, causing serialized LSP
round-trips; change this to run cleanAndValidateProject concurrently using
Promise.allSettled over an array of promises created from projectPaths (keep
using extension.ballerinaExtInstance.langClient and the existing
cleanAndValidateProject function), and before starting and/or inside each
cleanup promise check token.isCancellationRequested to short-circuit cancelled
runs; preserve the existing error-logging behavior (console.error) for failures
since allSettled will return results you can inspect and log.